Skip to content

(Closes #512) fix for circular import when importing fparser.two.Fortran2008#513

Merged
sergisiso merged 3 commits into
masterfrom
512_f2008_import_bug
Jun 19, 2026
Merged

(Closes #512) fix for circular import when importing fparser.two.Fortran2008#513
sergisiso merged 3 commits into
masterfrom
512_f2008_import_bug

Conversation

@arporter

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.24%. Comparing base (beffc28) to head (89facde).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #513   +/-   ##
=======================================
  Coverage   92.24%   92.24%           
=======================================
  Files          88       88           
  Lines       13866    13866           
=======================================
  Hits        12790    12790           
  Misses       1076     1076           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arporter

Copy link
Copy Markdown
Member Author

A tiny fix. The test was the difficult bit. Ready for review from @hiker, @sergisiso or @LonelyCat124.

@sergisiso sergisiso left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arporter Thanks for fixing this and adding the test. This is ready to merge.

@sergisiso sergisiso merged commit ea5c601 into master Jun 19, 2026
6 checks passed
@sergisiso sergisiso deleted the 512_f2008_import_bug branch June 19, 2026 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants